-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feat] #30 - Task POST API 개발 #31
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
너무 갓벽해서 코드리뷰할게 없네요.. 더 공부해와서 리뷰 남길게요...
public KeyResultNumberExceededException() { | ||
super(ErrorType.KEY_RESULT_NUMBER_EXCEEDED); | ||
super(ErrorType.ACTIVE_KEY_RESULT_NUMBER_EXCEEDED); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p4;
캬캬~ 위아래 한칸이요!
public TaskInvalidPositionException() { | ||
super(ErrorType.INVALID_TASK_ORDER); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p4;
여기두요 !
public TaskNumberExceededException() { | ||
super(ErrorType.ACTIVE_TASK_NUMBER_EXCEEDED); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p4;
또 찾아버렸네요..
🚀PullRequest🚀
📟 관련 이슈
💻 작업 내용
📝 리뷰 노트
📸API TEST SNAPSHOT📸
한칸 띄우고 생성하거나 이상한 idx가 들어올 경우
KeyResult 당 최대 3개의 개수를 초과하여 생성을 요청할 경우
생성되는 사진은 따로 올리지 않겠습니다 !!